Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding a plugin to calculate the virtual temperature (version 2) #2061

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

mo-philrelton
Copy link

Address: https://metoffice.atlassian.net/browse/EPPT-1965

This PR adds a plugin to calculate the virtual temperature based on the air temperature and humidity mixing ratio.

No CLI added due to the move to the DAG runner.

This is a replacement for the closed #2058 to avoid history rewrites and git configuration issues.

Testing:

Ran tests and they passed OK

Added new tests for the new feature(s)

CLA

If a new developer, signed up to CLA

@mo-philrelton mo-philrelton added the EPP PR's related to Enhancing Post Processing team label Dec 6, 2024
@mo-philrelton mo-philrelton changed the title Epp1965 virtualtemperature Adding a plugin to calculate the virtual temperature (version 2) Dec 6, 2024
Copy link

codecov bot commented Dec 6, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.40%. Comparing base (84a8944) to head (9454738).
Report is 49 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff            @@
##           master    #2061    +/-   ##
========================================
  Coverage   98.39%   98.40%            
========================================
  Files         124      134    +10     
  Lines       12212    13138   +926     
========================================
+ Hits        12016    12928   +912     
- Misses        196      210    +14     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@Katie-Howard Katie-Howard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you just need to change a line in test_VirtualTemperature to get the tests to work and then it should be fine 👍

@mo-philrelton mo-philrelton force-pushed the epp1965_virtualtemperature branch from 354d649 to a0b412f Compare December 9, 2024 10:06
Katie-Howard
Katie-Howard previously approved these changes Dec 9, 2024
Copy link
Contributor

@Katie-Howard Katie-Howard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm happy with these changes now 👍

mo-robertneal
mo-robertneal previously approved these changes Dec 11, 2024
Copy link

@mo-robertneal mo-robertneal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The code changes look good to me and the unit tests all passed.

Copy link
Contributor

@Katie-Howard Katie-Howard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The failing test has been resolved so I'm happy with this 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
EPP PR's related to Enhancing Post Processing team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants